Skip to content

feat: Add checkpoint at start of opreations#19

Merged
wangyb-A merged 3 commits intoaws:mainfrom
wangyb-A:main
Oct 2, 2025
Merged

feat: Add checkpoint at start of opreations#19
wangyb-A merged 3 commits intoaws:mainfrom
wangyb-A:main

Conversation

@wangyb-A
Copy link
Contributor

@wangyb-A wangyb-A commented Oct 2, 2025

  • Add checkpoint at start of operations for

    • wait
    • wait for condition
    • step
    • run in child context
  • Modify logic

    • step:
      • AT_MOST_ONCE_PER_RETRY, : go retry logic - if retry, do retries, if out of retries checkpoint FAIL and raise StepInterrupted and fail
      • AT_LEAST_ONCE_PER_RETRY: continues execution normally
    • wait:
      • suspend if it's not a brand new execution

Alex Wang added 3 commits October 1, 2025 13:07
- suspend execution of step and wait_for_condition when the checkpointed result is pending
- refactor suspend, create a new suspend module
- test cases
- Add checkpoint at start of the operations for
  - Step
  - Wait for condition
  - Wait
  - Run in child context

- Update logic
  - Wait: suspend execution if it's not a brand new execution
  - Step:
    - AT_MOST_ONCE_PER_RETRY, : go retry logic - if retry, do retries, if out of retries checkpoint FAIL and raise StepInterrupted and fail
    - AT_LEAST_ONCE_PER_RETRY: continues execution normally
- Add checkpoint at start of operations for
  - wait
  - wait for condition
  - step
  - run in child context

- Modify logic
  - step:
    - AT_MOST_ONCE_PER_RETRY, : go retry logic - if retry, do retries, if out of retries checkpoint FAIL and raise StepInterrupted and fail
    - AT_LEAST_ONCE_PER_RETRY: continues execution normally
  - wait:
    - suspend if it's not a brand new execution
@wangyb-A wangyb-A requested a review from yaythomas October 2, 2025 16:37
@wangyb-A wangyb-A marked this pull request as ready for review October 2, 2025 16:38
Copy link
Member

@yaythomas yaythomas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

commit history has duplicate commits... please be sure to squash

other than that looks good!

@wangyb-A wangyb-A merged commit c1bc794 into aws:main Oct 2, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants